Skip to content

Brigkit is now explosion resistant (take 2) - #5929

Open
Celestial3rror404 wants to merge 2 commits into
ss14Starlight:starlight-devfrom
Celestial3rror404:Blastproof-brigkit
Open

Brigkit is now explosion resistant (take 2)#5929
Celestial3rror404 wants to merge 2 commits into
ss14Starlight:starlight-devfrom
Celestial3rror404:Blastproof-brigkit

Conversation

@Celestial3rror404

@Celestial3rror404 Celestial3rror404 commented Sep 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Short description

Adds explosion resistance to the brigmedic's special medkit

Why we need to add this

A well received idea in https://discord.com/channels/1272545509562777621/1541772477758308434
As a security role, the brigmedic will often encounter explosions, which can cause them to lose a significant stock of chems in an instant.

Media (Video/Screenshots)

Before minibomb
Make Brigkit bombproof 1
After minibomb
Make Brigkit bombproof 2

Checks

  • I do not require assistance to complete the PR.
  • Before posting/requesting review of a PR, I have verified that the changes work.
  • I have added screenshots/videos of the changes, or this PR does not change in-game mechanics.
  • I affirm that my changes are licensed under the MIT License and grant permission for use in this repository under its conditions.

Changelog

🆑 Celestial3rror404

  • tweak: The Brigkit now features bombproof casing.

@Celestial3rror404
Celestial3rror404 requested a review from a team September 1, 2026 23:23
@github-actions github-actions Bot added size/XS S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. Changes: No C# Changes: Prototypes S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. and removed S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. labels Sep 1, 2026
@coderabbitai

coderabbitai Bot commented Sep 1, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Team

Run ID: 2d5f2fb1-3cc1-4502-9bee-61c15f48ef12

📥 Commits

Reviewing files that changed from the base of the PR and between d532506 and b8cc87e.

📒 Files selected for processing (1)
  • Resources/Prototypes/_Starlight/Entities/Objects/Specific/Medical/medical.yml

Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review.

📝 Summary

Summary

The Brigkit now has an ExplosionResistance component with damageCoefficient: 0. Explosions will not destroy the brigmedic’s chem supplies.

The change affects Resources/Prototypes/_Starlight/Entities/Objects/Specific/Medical/medical.yml and applies only to SecurityMedkit.

Risk areas

  • The Brigkit is fully immune to explosive damage.
  • This may affect balance when the Brigkit is used near explosions.

Guideline gaps

  • The PR context does not explain why the change belongs in the Starlight override instead of a Shared prototype.
  • The PR context does not include a Starlight ownership comment.
  • The PR context does not describe the design impact of making the Brigkit fully explosion-proof.

Walkthrough

The SecurityMedkit entity gains an ExplosionResistance component with damageCoefficient: 0.

Changes

Security medkit resistance

Layer / File(s) Summary
Add explosion resistance
Resources/Prototypes/_Starlight/Entities/Objects/Specific/Medical/medical.yml		 SecurityMedkit gains complete immunity to explosive damage.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to b8cc8

This localized change makes the Brigmedic’s special medkit immune to explosive damage. No actionable merge-blocking risk remains beyond normal checks and review.

Suggested reviewers: redmushie

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the main change: explosion resistance for the Brigkit. The “take 2” suffix is minor but does not reduce clarity.
Description check ✅ Passed The description explains the Brigkit change, its gameplay purpose, supporting discussion, screenshots, verification, and changelog entry.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Content + Integration Test Results

Tests 📝 Passed ✅ Failed ❌ Duration ⏱️
1697 1677 0 12m 27s

🎉 No failed tests in this run.

Github Test Reporter by CTRF 💚

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Changes: No C# Changes: Prototypes S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/XS

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Celestial3rror404